Skip to content

PR for #31 - MvvmLight support for navigation#33

Merged
tom-made merged 1 commit intomasterfrom
page-mvvm
May 22, 2018
Merged

PR for #31 - MvvmLight support for navigation#33
tom-made merged 1 commit intomasterfrom
page-mvvm

Conversation

@jamesmcroft
Copy link
Copy Markdown
Member

Change Title

Added a new project for MvvmLight components for navigation. Includes a basic page which passes the navigation events off to a common backing view-model.

Also includes bug fixes for an issue with the navigation argument passed to the pages when navigating and navigated from. This is now inline with Windows functionality where it will pass the argument for the page that you're navigating to when navigating from and navigated from.

Associated Issues

#31 - [Feature] MvvmLight support for navigation

Added MvvmLight project for navigation including an MvvmPage and a base view model for handling navigation events.

Updated samples to test functionality and added bug fixes to the Android navigation frame to be in line with UWP navigation.
@jamesmcroft jamesmcroft requested a review from tom-made May 22, 2018 12:48
Copy link
Copy Markdown
Contributor

@tom-made tom-made left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy with the changes here. Won't close off issue until support for iOS.

@tom-made tom-made merged commit 2d064a2 into master May 22, 2018
@tom-made tom-made deleted the page-mvvm branch May 22, 2018 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants